Skip to content

feat(firestore): Add support for subqueries#7839

Merged
dlarocque merged 27 commits intomainfrom
dl/subq
Apr 8, 2026
Merged

feat(firestore): Add support for subqueries#7839
dlarocque merged 27 commits intomainfrom
dl/subq

Conversation

@dlarocque
Copy link
Copy Markdown
Contributor

@dlarocque dlarocque commented Mar 20, 2026

Adds support for Pipeline subqueries.

Ported from: firebase/firebase-android-sdk#7736

API Proposal: go/firestore-pipelines-subquery

@product-auto-label product-auto-label bot added the api: firestore Issues related to the Firestore API. label Mar 20, 2026
@dlarocque dlarocque force-pushed the dl/subq branch 2 times, most recently from 850ba24 to dd1a205 Compare March 23, 2026 20:07
Base automatically changed from dl/defer-validation to main March 24, 2026 14:12
@dlarocque dlarocque marked this pull request as ready for review March 26, 2026 20:01
@dlarocque dlarocque requested a review from a team as a code owner March 26, 2026 20:01
Copy link
Copy Markdown
Contributor

@cherylEnkidu cherylEnkidu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you add this test as well: https://github.com/googleapis/java-firestore/pull/2323/changes#diff-566df7e9507530a3a2b0c886b9408ac028ed2f741506e4ed33a3b4075ea36672R986

Also implement the check when calling subcollection in Union stage

@dlarocque dlarocque requested a review from cherylEnkidu April 2, 2026 15:55
Copy link
Copy Markdown
Contributor

@MarkDuckworth MarkDuckworth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with suggestions for ref doc cleanup

@MarkDuckworth MarkDuckworth added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 2, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 2, 2026
Copy link
Copy Markdown
Contributor

@MarkDuckworth MarkDuckworth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with some minor documentation improvements

@dlarocque dlarocque merged commit fb317f8 into main Apr 8, 2026
36 checks passed
@dlarocque dlarocque deleted the dl/subq branch April 8, 2026 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: firestore Issues related to the Firestore API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants